-
Notifications
You must be signed in to change notification settings - Fork 327
#673 Add ReasoningEffort.minimal value #700
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@jsquire @christothes @joseharriaga @trrwilson @ShivangiReja Hello everyone! This parameter is quite urgent for us, as it's crucial for our upcoming release. If anyone could fit in a quick review, it would be incredibly helpful. Thank you so much in advance for your time and assistance! ❤️ |
The |
3861db2
to
61540f5
Compare
@christothes I run this script and got this error
I install this library https://github.com/MigleSur/GenAPI And now got this
Can you help me, please? :) P.S. I'm a little confused because blast is something from Linux, although the scripts are for powershell (Windows) P.P.S. Okay, I realized that this repository is something completely different from what he asks for, but in any case, I didn’t find on the Internet how to replace it + I also didn’t see any script with this name in the repository |
6c8a3fd
to
c8e8f2d
Compare
Ok, I finally solve the problem :) |
Thanks very much for your contribution, @kurnakovv. Would you please be so kind as to add a change log entry to capture this? |
@christothes Thank you so much for your approval 👍 @jsquire Yes, no problem! I’ve updated the CHANGELOG file. |
#673 issue
Second try :)
Thanks so much for your help @jsquire and @christothes in #675
I fixed my mistake and ran this Invoke-CodeGen script.
P.S. I guess I shouldn't have included the changes in the

tspCodeModel.json
file, right? :))